Skip to content

Conversation

@ilayaperumalg
Copy link
Member

  • Rename the artifact ID of the chat memory repository artifacts:

    • spring-ai-model-chat-memory-jdbc -> spring-ai-model-chat-memory-repository-jdbc
    • spring-ai-model-chat-memory-cassandra -> spring-ai-model-chat-memory-repository-cassandra
    • spring-ai-model-chat-memory-neo4j -> spring-ai-model-chat-memory-repository-neo4j
  • Rename the package names to include "repository". Example: org.springframework.ai.chat.memory.repository.jdbc.JdbcChatMemoryRepository

    • This package renaming also requires to change the default schema location for the jdbc repository to include "repository" - Update the docs
  • Update the artifact IDs in the parent POM, BOM, autoconfiguration and starters

  • Update upgrade notes and docs to describe the changes

 - Rename the artifact ID of the chat memory repository artifacts:

    - `spring-ai-model-chat-memory-jdbc` -> `spring-ai-model-chat-memory-repository-jdbc`
    - `spring-ai-model-chat-memory-cassandra` -> `spring-ai-model-chat-memory-repository-cassandra`
    - `spring-ai-model-chat-memory-neo4j` -> `spring-ai-model-chat-memory-repository-neo4j`

 - Rename the package names to include "repository". Example: org.springframework.ai.chat.memory.repository.jdbc.JdbcChatMemoryRepository
    - This package renaming also requires to change the default schema location for the jdbc repository to include "repository"
      - Update the docs

 - Update the artifact IDs in the parent POM, BOM, autoconfiguration and starters
 - Update upgrade notes and docs to describe the changes

Signed-off-by: Ilayaperumal Gopinathan <[email protected]>
 - Make sure to set the dialect via datasource

Signed-off-by: Ilayaperumal Gopinathan <[email protected]>
@ilayaperumalg ilayaperumalg force-pushed the chat-memory-repository-refactoring branch from 50ccfd6 to 54832fe Compare May 12, 2025 16:28
@markpollack
Copy link
Member

merged in 2d517ee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants